Skip to content

Conversation

BuonOmo
Copy link
Collaborator

@BuonOmo BuonOmo commented Oct 9, 2025

TODOs:

@BuonOmo BuonOmo force-pushed the feat/rails-8-1 branch 2 times, most recently from cb4869f to d8e3e07 Compare October 9, 2025 11:36
When using `--fail-fast` option the rails reporter could raise before
other reporters records the failure. Hence we place rails last.

Signed-off-by: Ulysse Buonomo <[email protected]>
This test was failing due to missing a column in the model. We fix it by
resetting the cache after adding the column.

Signed-off-by: Ulysse Buonomo <[email protected]>
This test was failing due to missing a column in the model. We fix it by
resetting the cache after adding the column.

Signed-off-by: Ulysse Buonomo <[email protected]>
It was using old and invalid SQL statements. I took that
opportunity to also change a bit its behaviour:
- Remove the related rake task and automatically cache
  the schema if non-existent.
- Cache every databases used in tests.
- Ensure that cache name is based on schema files.
Rails is moving towards ractor compatibility, and to
do so it swapped to frozen database objects. Hence
we cannot simply memoize the factories anymore. I've
checked the initialization speed for factories, and
the memoization was too fast for it to be worth
implementing a cache.

See rails/rails@d2da816
See rails/rails@76448a0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant